Skip to content

Improve docker build#294

Open
orgads wants to merge 1 commit into
drachtio:mainfrom
orgads:improve-docker
Open

Improve docker build#294
orgads wants to merge 1 commit into
drachtio:mainfrom
orgads:improve-docker

Conversation

@orgads

@orgads orgads commented Jun 24, 2023

Copy link
Copy Markdown
Contributor
  • Use current code instead of cloning from github.
  • Use multi-stage instead of deleting everything.

@orgads orgads force-pushed the improve-docker branch 2 times, most recently from b103eea to 4083778 Compare June 29, 2023 12:54
@davehorton

Copy link
Copy Markdown
Collaborator

Hmm this fails for me.

Also, I don't like the fact that now we have lost the ability to see the specific git tag that this was built from.

g++ -DHAVE_CONFIG_H -I. -I.. <...> -DDRACHTIO_VERSION=\"1.0.0\"

@orgads

orgads commented Jun 29, 2023

Copy link
Copy Markdown
Contributor Author

The failure is:

Killed signal terminated program cc1plus

I don't see how that's related to any of my changes.

@orgads orgads force-pushed the improve-docker branch 2 times, most recently from 88393b1 to fc0ad4b Compare June 29, 2023 15:21
@orgads

orgads commented Jun 29, 2023

Copy link
Copy Markdown
Contributor Author

How about now? You can build with ./build-docker.sh, and it should use the current version.

@orgads

orgads commented Aug 21, 2023

Copy link
Copy Markdown
Contributor Author

Rebased. @davehorton ping.

@orgads

orgads commented Jan 15, 2025

Copy link
Copy Markdown
Contributor Author

ping

* Use current code instead of cloning from github.
* Use multi-stage instead of deleting everything.
@orgads

orgads commented Jan 19, 2025

Copy link
Copy Markdown
Contributor Author

Rebased

@orgads

orgads commented Feb 2, 2025

Copy link
Copy Markdown
Contributor Author

@davehorton

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants